Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert brand store snaps to TypeScript #4312

Merged
merged 1 commit into from
Jun 28, 2023

Conversation

steverydz
Copy link
Contributor

@steverydz steverydz commented Jun 28, 2023

Done

Converted the brand store snaps to TypeScript

How to QA

Issue / Card

Fixes https://warthogs.atlassian.net/browse/WD-4853

@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

Merging #4312 (04147a1) into main (1ef6be4) will decrease coverage by 16.05%.
The diff coverage is n/a.

❗ Current head 04147a1 differs from pull request most recent head 3e46145. Consider uploading reports for the commit 3e46145 to get more accurate results

@@             Coverage Diff             @@
##             main    #4312       +/-   ##
===========================================
- Coverage   87.56%   71.52%   -16.05%     
===========================================
  Files          95       90        -5     
  Lines        6653     2975     -3678     
  Branches        0      770      +770     
===========================================
- Hits         5826     2128     -3698     
+ Misses        827      726      -101     
- Partials        0      121      +121     
Flag Coverage Δ
javascript 71.52% <ø> (?)
python ?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 185 files with indirect coverage changes

@@ -27,24 +27,38 @@ import SnapsSearch from "./SnapsSearch";
import SectionNav from "../SectionNav";
import StoreNotFound from "../StoreNotFound";

import type { BrandStores, Snap, Snaps, Members } from "../../types/shared";

// type Snaps = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y u do comment?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Lukewh Removed

@steverydz steverydz force-pushed the convert-snaps-to-typescript branch from 4aba6d3 to 3e46145 Compare June 28, 2023 15:32
Copy link
Contributor

@Lukewh Lukewh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM

@steverydz steverydz merged commit 953b23c into canonical:main Jun 28, 2023
8 checks passed
@steverydz steverydz deleted the convert-snaps-to-typescript branch June 28, 2023 15:57
Copy link
Contributor

@Lukewh Lukewh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants